refactor!: upgrade SDK to A2A 1.0 specs #572

muscariello · 2025-11-28T21:00:39Z

Summary

This PR migrates the a2a-python SDK from Pydantic-based types to protobuf-generated types, completing the upgrade to A2A v1.0. Fixes #559

Breaking Changes

Replace Pydantic-based type system with protobuf types from a2a_pb2
Update Part usage from Part(root=TextPart(text=...)) to Part(text=...)
Update Role enum from Role.user/Role.agent to Role.ROLE_USER/Role.ROLE_AGENT
Update TaskState enum to use TASK_STATE_* prefix

Changes

Update all source files to use proto types directly from a2a_pb2
Replace model_dump() with MessageToDict() for JSON serialization
Replace model_copy(deep=True) with CopyFrom() for proto cloning
Add new types module with proto imports and SDK-specific extras
Add proto_utils module with identity conversion utilities
Fix REST handler resource name formats for push notification configs
Fix gRPC handler to use SubscribeToTask instead of TaskSubscription
Fix database task store to handle proto objects from ORM
Update all test files for proto patterns and assertions

Testing

601 tests passing
23 tests skipped (expected - require DB connections or cryptography deps)

BREAKING CHANGE: Replace Pydantic-based type system with protobuf types - Update all source files to use proto types directly from a2a_pb2 - Replace model_dump() with MessageToDict() for JSON serialization - Replace model_copy(deep=True) with CopyFrom() for proto cloning - Update Part usage from Part(root=TextPart(...)) to Part(text=...) - Update Role enum from Role.user to Role.ROLE_USER - Update TaskState enum to use TASK_STATE_* prefix - Add new types module with proto imports and SDK-specific extras - Add proto_utils module with identity conversion utilities - Fix REST handler resource name formats for push notification configs - Fix gRPC handler to use SubscribeToTask instead of TaskSubscription - Fix database task store to handle proto objects from ORM - Update all test files for proto patterns and assertions Tested: 601 tests passing, 23 skipped (expected - DB/crypto deps) Signed-off-by: Luca Muscariello <[email protected]>

gemini-code-assist · 2025-11-28T21:02:22Z

Summary of Changes

Hello @muscariello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request represents a foundational shift in the SDK's type system, moving from Pydantic models to native protobuf-generated types. This change aligns the SDK more closely with the A2A v1.0 specification, enhancing type consistency and performance. The migration necessitated extensive updates across the codebase, impacting how data is structured, serialized, and handled throughout the client and server components. While introducing several breaking changes to the API, the goal is to provide a more robust and future-proof foundation for A2A interactions.

Highlights

Protobuf Type Migration: The entire SDK has been migrated from Pydantic-based types to protobuf-generated types, completing the upgrade to A2A v1.0. This involves significant changes across client and server components to directly utilize protobuf messages.
Legacy Client Removal: The deprecated A2AClient (JSON-RPC) and A2AGrpcClient (gRPC) legacy client wrappers have been removed, streamlining the client API. Users should now use ClientFactory to create clients with specific transports.
API Changes and Breaking Changes: Key API changes include updating Part usage from Part(root=TextPart(text=...)) to Part(text=...), renaming Role enum values (e.g., Role.user to Role.ROLE_USER), and updating TaskState enum values with a TASK_STATE_* prefix. Method signatures across client transports and handlers have been updated to reflect the new protobuf request/response types.
Serialization and Deserialization: The internal serialization logic has been updated to use MessageToDict() for JSON serialization and ParseDict() for deserialization, replacing Pydantic's model_dump() and model_validate() methods. This ensures compatibility with protobuf messages.
Build and Configuration Updates: The buf.gen.yaml configuration has been updated to generate protobuf types into src/a2a/types instead of src/a2a/grpc. pyproject.toml and related linting/type-checking configurations (mypy, ruff) have been adjusted to exclude the new protobuf-generated files in their new location and to include the new json-rpc dependency.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This is a massive and well-executed refactoring to migrate the SDK from Pydantic types to protobuf-generated types, completing the upgrade to A2A v1.0. The changes are extensive, touching almost every part of the SDK, from client and server logic to transports, handlers, database models, and tests.

My review finds that the migration has been executed thoroughly and consistently. Key changes include:

Replacing Pydantic models with protobuf messages from a2a_pb2.
Updating data serialization from model_dump() to MessageToDict() and ParseDict().
Refactoring client transports (grpc, jsonrpc, rest) to work directly with protobuf types, which simplifies the code significantly.
Updating method signatures and data structures across the client and server components.
Introducing a new types module as a central place for all type definitions.
Removing legacy client implementations.

The overall quality of the changes is high. The code is now more aligned with the A2A v1.0 specification, and the gRPC transport, in particular, is much cleaner. The test suite has been updated to reflect these changes, with a large number of tests passing.

I have a few suggestions for improving code maintainability and simplifying the logic in the AuthInterceptor and the database task store. Otherwise, the PR looks excellent.

gemini-code-assist · 2025-11-28T21:05:37Z

src/a2a/server/tasks/database_task_store.py

    def _to_orm(self, task: Task) -> TaskModel:
-        """Maps a Pydantic Task to a SQLAlchemy TaskModel instance."""
+        """Maps a Proto Task to a SQLAlchemy TaskModel instance."""
+        # Convert proto to dict for storing in JSON columns
+        task_dict = MessageToDict(task, preserving_proto_field_name=True)
        return self.task_model(
            id=task.id,
            context_id=task.context_id,
-            kind=task.kind,
-            status=task.status,
-            artifacts=task.artifacts,
-            history=task.history,
-            task_metadata=task.metadata,
+            kind='task',  # Default kind for tasks
+            status=task_dict.get('status'),
+            artifacts=task_dict.get('artifacts', []),
+            history=task_dict.get('history', []),
+            task_metadata=task_dict.get('metadata'),
        )


This function can be simplified. Instead of converting the entire Task protobuf message to a dictionary, you can pass the nested protobuf message objects (status, artifacts, history) directly to the TaskModel constructor. The PydanticType and PydanticListType decorators will handle their serialization. This avoids the overhead of MessageToDict for the whole task object.

def _to_orm(self, task: Task) -> TaskModel: """Maps a Proto Task to a SQLAlchemy TaskModel instance.""" return self.task_model( id=task.id, context_id=task.context_id, kind='task', # Default kind for tasks status=task.status, artifacts=list(task.artifacts), history=list(task.history), task_metadata=MessageToDict(task.metadata) if task.metadata.fields else None, )

gemini-code-assist · 2025-11-28T21:05:37Z

src/a2a/server/tasks/database_task_store.py

    def _from_orm(self, task_model: TaskModel) -> Task:
-        """Maps a SQLAlchemy TaskModel to a Pydantic Task instance."""
-        # Map database columns to Pydantic model fields
+        """Maps a SQLAlchemy TaskModel to a Proto Task instance."""
+        # The ORM columns return proto objects for status, artifacts, history
+        # We need to convert them back to dicts for ParseDict
        task_data_from_db = {
            'id': task_model.id,
            'context_id': task_model.context_id,
-            'kind': task_model.kind,
-            'status': task_model.status,
-            'artifacts': task_model.artifacts,
-            'history': task_model.history,
-            'metadata': task_model.task_metadata,  # Map task_metadata column to metadata field
        }
-        # Pydantic's model_validate will parse the nested dicts/lists from JSON
-        return Task.model_validate(task_data_from_db)
+        # Add status if present (already a proto object from PydanticType)
+        if task_model.status is not None:
+            task_data_from_db['status'] = MessageToDict(task_model.status, preserving_proto_field_name=True)
+        # Add artifacts if present (list of proto objects)
+        if task_model.artifacts:
+            task_data_from_db['artifacts'] = [
+                MessageToDict(a, preserving_proto_field_name=True) if hasattr(a, 'DESCRIPTOR') else a
+                for a in task_model.artifacts
+            ]
+        # Add history if present (list of proto objects)
+        if task_model.history:
+            task_data_from_db['history'] = [
+                MessageToDict(m, preserving_proto_field_name=True) if hasattr(m, 'DESCRIPTOR') else m
+                for m in task_model.history
+            ]
+        # Add metadata if present
+        if task_model.task_metadata is not None:
+            task_data_from_db['metadata'] = task_model.task_metadata
+        # Use ParseDict to create proto from dict
+        return ParseDict(task_data_from_db, Task())


This function seems overly complex. Since PydanticType and PydanticListType already deserialize the JSON from the database into protobuf message objects, you can construct the final Task protobuf object directly using these objects, rather than converting them back to dictionaries and then parsing the whole dictionary again. This would be more efficient and easier to read.

def _from_orm(self, task_model: TaskModel) -> Task: """Maps a SQLAlchemy TaskModel to a Proto Task instance.""" task = Task( id=task_model.id, context_id=task_model.context_id, ) if task_model.status: task.status.CopyFrom(task_model.status) if task_model.artifacts: task.artifacts.extend(task_model.artifacts) if task_model.history: task.history.extend(task_model.history) if task_model.task_metadata: task.metadata.update(task_model.task_metadata) return task

gemini-code-assist · 2025-11-28T21:05:37Z

src/a2a/client/auth/interceptor.py

+    which = scheme.WhichOneof('scheme')
+    if which == 'api_key_security_scheme':
+        return scheme.api_key_security_scheme
+    elif which == 'http_auth_security_scheme':
+        return scheme.http_auth_security_scheme
+    elif which == 'oauth2_security_scheme':
+        return scheme.oauth2_security_scheme
+    elif which == 'open_id_connect_security_scheme':
+        return scheme.open_id_connect_security_scheme
+    elif which == 'mtls_security_scheme':
+        return scheme.mtls_security_scheme
+    return None


This function can be simplified by using getattr to dynamically access the oneof field. This makes the code more concise and easier to maintain if more security schemes are added in the future.

which = scheme.WhichOneof('scheme') if which: return getattr(scheme, which) return None

- Fix agent_app.py Part access pattern for proto (HasField/direct access) - Fix ClientEvent handling in E2E tests (StreamResponse, not Task) - Fix notifications_app.py to serialize proto Task to dict - Update SetTaskPushNotificationConfigRequest interface in handlers - Update default_request_handler to use request.parent instead of name - Update jsonrpc_handler to pass full request to handler - Update unit tests to use SetTaskPushNotificationConfigRequest Signed-off-by: Luca Muscariello <[email protected]>

Tehsmash · 2025-12-01T10:07:00Z

buf.gen.yaml

 inputs:
  - git_repo: https://github.com/a2aproject/A2A.git
-    ref: main
+    ref: transports


We need to change this back to "main" to pick up the latest fixes from the a2a.proto and before we merge we need to switch to the tag for v1.0 (when its cut)

Tehsmash · 2025-12-01T10:27:15Z

src/a2a/client/auth/interceptor.py

-                    scheme_def = scheme_def_union.root
+                    scheme_def = _get_security_scheme_value(scheme_def_union)
+                    if not scheme_def:
+                        continue


Personal preference would be to use the functions available from proto to do the matching below instead of ending up with a non-statically typed function here. E.g.

match scheme.WhichOneof('scheme'): case 'http_auth_security_scheme' if scheme.http_auth_security_scheme.lower() == 'bearer': ...

tbh I think it would read better as a set of if blocks:

scheme = agent_card.security_schemes.get(scheme_name) if scheme.HasField('http_auth_security_scheme') and scheme.http_auth_security_scheme.lower() == 'bearer': ... if scheme.HasField('oauth2_security_scheme') or scheme.HasField('open_id_connect_security_scheme'): ...

Not sure how's HasField improves things here.

As explained our call, its mostly about readability, but also prevent us using reflection to check the python types, and instead we can just rely on the boolean check of "does the request have this field set" then it can use it.

Check now @Tehsmash

I don't seem to be able to resolve my own threads on the a2a repos, but I'll leave a LGTM if I think it should be resolved.

Tehsmash · 2025-12-01T11:17:25Z

src/a2a/types/extras.py

+
+# Alias for backward compatibility - the proto uses SendMessageRequest
+# where old code might use MessageSendParams
+from a2a.types.a2a_pb2 import SendMessageRequest as MessageSendParams


Lets not do this, any code that is already using a2a.types.MessageSendParams would need change the imports anyway and I think this will be confusing.

this is a quick removal that in any case is supposed to be removed before the PR is complete.

Tehsmash · 2025-12-01T11:18:01Z

src/a2a/types/extras.py

+)
+
+# Alias for streaming - same as SendMessageRequest in the proto
+SendStreamingMessageRequest = SendMessageRequest = MessageSendParams


-1, lets just update the code to use SendMessageRequest.

Tehsmash · 2025-12-01T11:19:40Z

src/a2a/types/extras.py

+    """Transport protocol string constants for backward compatibility."""
+    jsonrpc = "JSONRPC"
+    http_json = "HTTP+JSON"
+    grpc = "GRPC"


In the client code I add these as constants, if we want to put them somewhere common I would add a "constants.py" / constants module outside of "types".

To move this elsewhere I would have to solve a circular dependency first. There needs to be additional refactoring work before I can do that.

I am moving things from the refactor out of extras.py until it is empty. Then I will remove the file. A few more commits I think.

Tehsmash · 2025-12-01T11:22:40Z

src/a2a/types/extras.py

+# Union of all A2A error types
+A2AError = Union[
+    JSONRPCError,
+    JSONParseError,
+    InvalidRequestError,
+    MethodNotFoundError,
+    InvalidParamsError,
+    InternalError,
+    TaskNotFoundError,
+    TaskNotCancelableError,
+    PushNotificationNotSupportedError,
+    UnsupportedOperationError,
+    ContentTypeNotSupportedError,
+    InvalidAgentResponseError,
+    AuthenticatedExtendedCardNotConfiguredError,
+]
+
+
+class JSONRPCRequest(A2ABaseModel):
+    """Represents a JSON-RPC 2.0 Request object."""
+
+    jsonrpc: Literal["2.0"] = "2.0"
+    method: str
+    params: Any | None = None
+    id: str | int | None = None
+
+
+class JSONRPCResponse(A2ABaseModel):
+    """Represents a JSON-RPC 2.0 Success Response object."""
+
+    jsonrpc: Literal["2.0"] = "2.0"
+    result: Any
+    id: str | int | None = None
+
+
+class JSONRPCErrorResponse(A2ABaseModel):
+    """Represents a JSON-RPC 2.0 Error Response object."""
+
+    jsonrpc: Literal["2.0"] = "2.0"
+    error: A2AError
+    id: str | int | None = None
+
+
+# Type alias for A2A requests (union of all request types)
+# This maps to the various request message types in the proto
+from a2a.types.a2a_pb2 import (
+    CancelTaskRequest,
+    GetExtendedAgentCardRequest,
+    GetTaskPushNotificationConfigRequest,
+    GetTaskRequest,
+    SendMessageRequest,
+    SetTaskPushNotificationConfigRequest,
+    SubscribeToTaskRequest,
+)
+
+A2ARequest = Union[
+    SendMessageRequest,
+    GetTaskRequest,
+    CancelTaskRequest,
+    SetTaskPushNotificationConfigRequest,
+    GetTaskPushNotificationConfigRequest,
+    SubscribeToTaskRequest,
+    GetExtendedAgentCardRequest,
+]
+
+
+# JSON-RPC Success Response types
+# These wrap the result of successful RPC calls
+# Note: result is typed as Any to allow both proto messages and dicts
+class GetTaskSuccessResponse(A2ABaseModel):
+    """Success response for GetTask RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class CancelTaskSuccessResponse(A2ABaseModel):
+    """Success response for CancelTask RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class SendMessageSuccessResponse(A2ABaseModel):
+    """Success response for SendMessage RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class SendStreamingMessageSuccessResponse(A2ABaseModel):
+    """Success response for streaming message RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any  # Streaming events
+
+
+class SetTaskPushNotificationConfigSuccessResponse(A2ABaseModel):
+    """Success response for SetTaskPushNotificationConfig RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class GetTaskPushNotificationConfigSuccessResponse(A2ABaseModel):
+    """Success response for GetTaskPushNotificationConfig RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class ListTaskPushNotificationConfigSuccessResponse(A2ABaseModel):
+    """Success response for ListTaskPushNotificationConfig RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any
+
+
+class DeleteTaskPushNotificationConfigSuccessResponse(A2ABaseModel):
+    """Success response for DeleteTaskPushNotificationConfig RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: None = None
+
+
+class GetAuthenticatedExtendedCardSuccessResponse(A2ABaseModel):
+    """Success response for GetAuthenticatedExtendedCard RPC."""
+    jsonrpc: Literal["2.0"] = "2.0"
+    id: str | int | None = None
+    result: Any  # AgentCard


Let not add our own implementation of this. In the client code I pull in the "jsonrpc" package from pypi, which has the JSONRPC20Response type already defined, then we only need to concern ourselves with populating the result or error fields.

Tehsmash · 2025-12-01T11:44:41Z

src/a2a/server/apps/jsonrpc/fastapi_app.py

+            # A2ARequest is now a Union type of proto messages, so we can't use
+            # model_json_schema. Instead, we just mark it as added without
+            # adding the schema since proto types don't have Pydantic schemas.
+            # The OpenAPI schema will still be functional for the endpoints.


LOL ... "the code doesn't support this any more so we'll just mark this test as passed anyway"

In all seriousness this should be replaced by the generated a2a.json schema I think.

I am not using a2a.json at all. So need to rethink about this test.

What I meant is that if fastapi wants to serve an OpenAPI schema, then we need to use the a2a.json instead of generating one from the python types.

Tehsmash · 2025-12-01T11:47:28Z

src/a2a/server/apps/jsonrpc/jsonrpc_app.py

+        'tasks/pushNotificationConfig/delete': DeleteTaskPushNotificationConfigRequest,
+        'tasks/resubscribe': TaskResubscriptionRequest,
+        'agent/authenticatedExtendedCard': GetExtendedAgentCardRequest,
    }


The method names have changed to align with the gRPC ones, i.e message/send is now "SendMessage". We need to fix that in the client and here.

I think I have fixed this everywhere now. I will keep checking...

Tehsmash · 2025-12-01T11:52:52Z

src/a2a/server/apps/jsonrpc/jsonrpc_app.py

    JSONRPCError,
    JSONRPCErrorResponse,
    JSONRPCRequest,
    JSONRPCResponse,


Lets pull in the jsonrpc dependency so that we can remove these from the types.

Tehsmash · 2025-12-01T12:01:04Z

src/a2a/server/apps/jsonrpc/jsonrpc_app.py

                    id=request_id, error=error
                )

        return self._create_response(context, handler_result)


I think we should revisit this code and match on the JSON RPC method first then perform the correct decoding as there are body types like "SendMessageRequest" which are reused for different methods.

For example:

def _process_request(self, request: JSONRPC20Request): match request.method: case "SendMessage": input: SendMessageRequest = ParseDict(request.params, SendMessageRequest()) result = await self.handler.on_send_message(input) case "CancelTask": input: CancelTaskRequest = ParseDict(request.params, CancelTaskRequest()) ... ...

I still think this would be cleaner than doing METHOD_TO_MODEL first and matching on the classes after the fact. And makes it more obvious IMO that the final / default case of the "match" would be "MethodNotFound".

Tehsmash · 2025-12-01T12:29:55Z

src/a2a/server/request_handlers/default_request_handler.py

        """
        # Create task manager and validate existing task
+        # Proto empty strings should be treated as None
+        task_id = params.request.task_id or None


Need to confirm if "task_id" should be just the UUID, or should be in the form "tasks/".
I think AIP for this said that it should be "tasks/" because this is the FQN for the task.

Tehsmash · 2025-12-01T15:03:11Z

src/a2a/utils/error_handlers.py

    TaskNotCancelableError,
    TaskNotFoundError,
    UnsupportedOperationError,
 )


We should move all these errors to errors.py instead of extras.

Tehsmash · 2025-12-01T15:09:42Z

src/a2a/utils/parts.py

        A list of dictionaries containing the data from any `DataPart` objects found.
    """
-    return [part.root.data for part in parts if isinstance(part.root, DataPart)]
+    from google.protobuf.json_format import MessageToDict


No need for an inline import, lets move this to the top of the file.

Tehsmash · 2025-12-01T15:10:46Z

src/a2a/utils/proto_utils.py

We can remove this file and its classes/functions, as we're using the proto types everywhere now.

Most of the stuff from here is gone, i think we can nuke the helper function here if we fix the response from the default_request_handler.

Tehsmash · 2025-12-01T15:11:53Z

src/a2a/utils/task.py

    for part in request.parts:
-        if isinstance(part.root, TextPart) and not part.root.text:
-            raise ValueError('TextPart content cannot be empty')
+        if part.text is not None and not part.text:


Suggested change

if part.text is not None and not part.text:

if part.HasField('text') and not part.text:

Tehsmash · 2025-12-01T15:16:11Z

src/a2a/utils/task.py

+        del task_copy.history[:]
+        task_copy.history.extend(limited_history)


Suggested change

del task_copy.history[:]

task_copy.history.extend(limited_history)

task_copy.history[:] = limited_history

- Fix E402: Move telemetry import to top in default_request_handler.py - Fix TRY300/RET504: Return directly in grpc_handler.py try blocks - Fix TRY004: Add noqa for valid ValueError in database_push_notification_config_store.py - Fix pyright: Add else branch for unbound client_event in base_client.py - Fix pyright: Add cast for rpc_request.data in jsonrpc.py transport All linter checks now pass: - ruff check: 0 errors - ruff format: 78 files formatted - mypy: 0 errors in 78 files - pyright: 0 errors, 0 warnings All 730 tests pass (including PostgreSQL and MySQL database tests)

Tehsmash · 2025-12-01T15:24:44Z

src/a2a/client/auth/interceptor.py

+

-def _get_security_scheme_value(scheme: SecurityScheme):
+def _get_security_scheme_value(scheme: SecurityScheme) -> _SecuritySchemeValue:


I think this change has just proven my point about moving this inline in the function below and changing how the "match" is done.

Tehsmash · 2025-12-01T15:58:23Z

src/a2a/server/tasks/task_manager.py

                task.history.append(task.status.message)
            if event.metadata:
-                task.metadata.update(event.metadata)
+                task.metadata.update(dict(event.metadata))  # type: ignore[arg-type]


We should use task.metadata.MergeFrom/CopyFrom here depending on the desired behaviour

…ry directly - Update JSONRPCHandler to return dict[str, Any] instead of Pydantic RootModels - Update response_helpers to build dicts with JSON-RPC 2.0 structure - Remove unused Pydantic response types from types module - Fix proto dependency loading in a2a_pb2.py - Update all tests to check dict responses instead of Pydantic models - Add TransportProtocol constants to utils module

…iases - Rename on_resubscribe_to_task to on_subscribe_to_task across all handlers - Update METHOD_TO_MODEL with gRPC-style method names (SendMessage, GetTask, etc.) - Update JSON-RPC client to use new method names - Fix ListTaskPushNotificationConfigResponse to use 'configs' field - Remove TaskResubscriptionRequest alias from extras.py - Update TransportProtocol imports to use a2a.utils.constants - Fix on_get_task_push_notification_config params type - Update all tests for new method names and response types

…s.py - Moved all error type definitions (JSONRPCError, A2AError, etc.) to utils/errors.py - Updated all imports across 15 files to use a2a.utils.errors - Defined A2ARequest union inline in types/__init__.py - Deleted the now-redundant extras.py file - Re-exported error types from types/__init__.py for public API

Tehsmash · 2025-12-02T15:00:39Z

src/a2a/client/auth/interceptor.py

-            or agent_card.security is None
-            or agent_card.security_schemes is None
+            or not agent_card.security
+            or not agent_card.security_schemes


Double checking the protobuf docs https://protobuf.dev/reference/python/python-generated/#embedded_message it sounds like we may need to use agent_card.HasField("...") this separates the difference between agent_card.security set, not set and set but empty from what I can tell.

Tehsmash · 2025-12-02T15:02:14Z

src/a2a/client/auth/interceptor.py

-                    scheme_def = scheme_def_union.root
+                    scheme_def = _get_security_scheme_value(scheme_def_union)
+                    if not scheme_def:
+                        continue


I don't seem to be able to resolve my own threads on the a2a repos, but I'll leave a LGTM if I think it should be resolved.

Tehsmash · 2025-12-02T15:09:12Z

src/a2a/utils/errors.py

+class A2ABaseModel(BaseModel):
+    """Base model for all A2A SDK types."""
+
+    model_config = {
+        'extra': 'allow',
+        'populate_by_name': True,
+        'arbitrary_types_allowed': True,
+    }
+
+
+# JSON-RPC Error types - A2A specific error codes
+class JSONRPCError(A2ABaseModel):
+    """Represents a JSON-RPC 2.0 Error object."""
+
+    code: int
+    """A number that indicates the error type that occurred."""
+    message: str
+    """A string providing a short description of the error."""
+    data: Any | None = None
+    """Additional information about the error."""
+
+
+class JSONParseError(A2ABaseModel):
+    """JSON-RPC parse error (-32700)."""
+
+    code: Literal[-32700] = -32700
+    message: str = 'Parse error'
+    data: Any | None = None
+
+
+class InvalidRequestError(A2ABaseModel):
+    """JSON-RPC invalid request error (-32600)."""
+
+    code: Literal[-32600] = -32600
+    message: str = 'Invalid Request'
+    data: Any | None = None
+
+
+class MethodNotFoundError(A2ABaseModel):
+    """JSON-RPC method not found error (-32601)."""
+
+    code: Literal[-32601] = -32601
+    message: str = 'Method not found'
+    data: Any | None = None
+
+
+class InvalidParamsError(A2ABaseModel):
+    """JSON-RPC invalid params error (-32602)."""
+
+    code: Literal[-32602] = -32602
+    message: str = 'Invalid params'
+    data: Any | None = None
+
+
+class InternalError(A2ABaseModel):
+    """JSON-RPC internal error (-32603)."""
+
+    code: Literal[-32603] = -32603
+    message: str = 'Internal error'
+    data: Any | None = None
+
+
+class TaskNotFoundError(A2ABaseModel):
+    """A2A-specific error for task not found (-32001)."""
+
+    code: Literal[-32001] = -32001
+    message: str = 'Task not found'
+    data: Any | None = None
+
+
+class TaskNotCancelableError(A2ABaseModel):
+    """A2A-specific error for task not cancelable (-32002)."""
+
+    code: Literal[-32002] = -32002
+    message: str = 'Task cannot be canceled'
+    data: Any | None = None
+
+
+class PushNotificationNotSupportedError(A2ABaseModel):
+    """A2A-specific error for push notification not supported (-32003)."""
+
+    code: Literal[-32003] = -32003
+    message: str = 'Push Notification is not supported'
+    data: Any | None = None
+
+
+class UnsupportedOperationError(A2ABaseModel):
+    """A2A-specific error for unsupported operation (-32004)."""
+
+    code: Literal[-32004] = -32004
+    message: str = 'This operation is not supported'
+    data: Any | None = None
+
+
+class ContentTypeNotSupportedError(A2ABaseModel):
+    """A2A-specific error for content type not supported (-32005)."""
+
+    code: Literal[-32005] = -32005
+    message: str = 'Incompatible content types'
+    data: Any | None = None
+
+
+class InvalidAgentResponseError(A2ABaseModel):
+    """A2A-specific error for invalid agent response (-32006)."""
+
+    code: Literal[-32006] = -32006
+    message: str = 'Invalid agent response'
+    data: Any | None = None
+
+
+class AuthenticatedExtendedCardNotConfiguredError(A2ABaseModel):
+    """A2A-specific error for authenticated extended card not configured (-32007)."""
+
+    code: Literal[-32007] = -32007
+    message: str = 'Authenticated Extended Card is not configured'
+    data: Any | None = None


The A2A exception's here shouldn't have JSONRPC specific data, the JSONRPC server/client components should have a map from the exceptions to the JSONRPC code and vice-versa for raising an exception on receipt of a JSONRPCError

JSONRPC specific exceptions like MethodNotFound should live in the JSONRPC implementation code or a separate JSONRPC common package IMO.

Tehsmash · 2025-12-02T15:09:28Z

src/a2a/utils/error_handlers.py

    TaskNotCancelableError,
    TaskNotFoundError,
    UnsupportedOperationError,
 )


Tehsmash · 2025-12-02T15:11:03Z

src/a2a/server/apps/jsonrpc/jsonrpc_app.py

+        'tasks/pushNotificationConfig/delete': DeleteTaskPushNotificationConfigRequest,
+        'tasks/resubscribe': TaskResubscriptionRequest,
+        'agent/authenticatedExtendedCard': GetExtendedAgentCardRequest,
    }


Tehsmash · 2025-12-02T16:19:38Z

src/a2a/server/request_handlers/request_handler.py

    async def on_resubscribe_to_task(
        self,
-        params: TaskIdParams,
+        params: CancelTaskRequest,


Tehsmash · 2025-12-02T16:20:41Z

src/a2a/server/models.py

+        if isinstance(self.pydantic_type, type) and issubclass(self.pydantic_type, ProtoMessage):
+            return ParseDict(value, self.pydantic_type())  # type: ignore[return-value]
+        # Assume it's a Pydantic model
+        return self.pydantic_type.model_validate(value)  # type: ignore[union-attr]


I think this is still relevant to models.py even though its been disconnected. You can resolve this and I'll open a new comment.

Tehsmash · 2025-12-02T16:20:54Z

src/a2a/server/models.py

No need to support Pydantic here IMO, just need to support the protobuf types.

One thought: We need to understand if we need to support old messages/tasks stored in the DB and translate them to the protobuf types. Otherwise we may require people to drop the DBs for their Agents

Tehsmash · 2025-12-02T16:21:56Z

src/a2a/utils/parts.py

        A list of dictionaries containing the data from any `DataPart` objects found.
    """
-    return [part.root.data for part in parts if isinstance(part.root, DataPart)]
+    from google.protobuf.json_format import MessageToDict


Tehsmash · 2025-12-02T16:26:35Z

src/a2a/utils/proto_utils.py

Most of the stuff from here is gone, i think we can nuke the helper function here if we fix the response from the default_request_handler.

# Description Refine error management for the streaming operation. Previously, errors were converted into stream parts, resulting in the loss of status info. The updated logic now first verifies if the request was successful; if it failed, a client error is returned, preserving the relevant status information. - [x] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [x] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - Important Prefixes for [release-please](https://github.com/googleapis/release-please): - `fix:` which represents bug fixes, and correlates to a [SemVer](https://semver.org/) patch. - `feat:` represents a new feature, and correlates to a SemVer minor. - `feat!:`, or `fix!:`, `refactor!:`, etc., which represent a breaking change (indicated by the `!`) and will result in a SemVer major. - [x] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [x] Appropriate docs were updated (if necessary) Fixes a2aproject#502 🦕

🤖 I have created a release *beep* *boop* --- ## [0.3.20](a2aproject/a2a-python@v0.3.19...v0.3.20) (2025-12-03) ### Bug Fixes * Improve streaming errors handling ([a2aproject#576](a2aproject#576)) ([7ea7475](a2aproject@7ea7475)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

muscariello · 2025-12-04T08:01:43Z

@Tehsmash I am going to wait a few more days for the A2A 1.0 specs to get the most recent updates.

# Description Just fixing various typos discovered while reading code of the repo: see commit diffs for details Cheers Didier - [X] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [X] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - [X] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [X] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> 🦕 N/A --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

…ect#581) This PR introduces digital signatures for Agent Cards to ensure authenticity and integrity, adhering to the A2A specification for [Agent Card Signing (Section 8.4).](https://a2a-protocol.org/latest/specification/#84-agent-card-signing) ## Changes: - Implement `Canonicalization` Logic (`src/a2a/utils/signing.py`) - Add `Signing` and `Verification` Utilities (`src/a2a/utils/signing.py`): - `create_agent_card_signer` which generates an `agent_card_signer` for signing `AgentCards` - `create_signature_verifier` which generates a `signature_verifier` for verification of `AgentCard` signatures - Enable signature verification support for `json-rpc`, `rest` and `gRPC` transports - Add Protobuf Conversion for Signatures (`src/a2a/utils/proto_utils.py`) ensuring `AgentCardSignature` can be serialized and deserialized for gRPC transport - Add related tests: - integration tests for fetching signed cards from the Server - unit tests for signing util - unit tests for protobuf conversions - [x] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [x] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - Important Prefixes for [release-please](https://github.com/googleapis/release-please): - `fix:` which represents bug fixes, and correlates to a [SemVer](https://semver.org/) patch. - `feat:` represents a new feature, and correlates to a SemVer minor. - `feat!:`, or `fix!:`, `refactor!:`, etc., which represent a breaking change (indicated by the `!`) and will result in a SemVer major. - [x] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [x] Appropriate docs were updated (if necessary) Release-As: 0.3.21

# Description Read further and discovered this additional (and final) set of typos

🤖 I have created a release *beep* *boop* --- ## [0.3.21](a2aproject/a2a-python@v0.3.20...v0.3.21) (2025-12-12) ### Documentation * Fixing typos ([a2aproject#586](a2aproject#586)) ([5fea21f](a2aproject@5fea21f)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

…ject#594) # Description This change allows passing custom `task_id_generator` and `context_id_generator` functions to the `SimpleRequestContextBuilder`. This provides flexibility in how task and context IDs are generated, defaulting to the previous behavior if no generators are provided. Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly: - [x] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [x] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - Important Prefixes for [release-please](https://github.com/googleapis/release-please): - `fix:` which represents bug fixes, and correlates to a [SemVer](https://semver.org/) patch. - `feat:` represents a new feature, and correlates to a SemVer minor. - `feat!:`, or `fix!:`, `refactor!:`, etc., which represent a breaking change (indicated by the `!`) and will result in a SemVer major. - [x] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [x] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> 🦕

# Description Adding 2 more tests to user.py to improve build code coverage - [X] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [X] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - [X] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [N/A] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> 🦕 N/A --------- Co-authored-by: Lukasz Kawka <[email protected]>

# Description Adding 21 tests for client/card_resolver.py They all pass: ``` ========================= test session starts ============================== collecting ... collected 21 items tests/client/test_card_resolver.py::TestA2ACardResolverInit::test_init_with_defaults PASSED [ 4%] tests/client/test_card_resolver.py::TestA2ACardResolverInit::test_init_with_custom_path PASSED [ 9%] tests/client/test_card_resolver.py::TestA2ACardResolverInit::test_init_strips_leading_slash_from_agent_card_path PASSED [ 14%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_success_default_path PASSED [ 19%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_success_custom_path PASSED [ 23%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_strips_leading_slash_from_relative_path PASSED [ 28%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_with_http_kwargs PASSED [ 33%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_root_path PASSED [ 38%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_http_status_error PASSED [ 42%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_json_decode_error PASSED [ 47%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_request_error PASSED [ 52%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_validation_error PASSED [ 57%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_logs_success PASSED [ 61%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_none_relative_path PASSED [ 66%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_empty_string_relative_path PASSED [ 71%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_different_status_codes[400] PASSED [ 76%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_different_status_codes[401] PASSED [ 80%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_different_status_codes[403] PASSED [ 85%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_different_status_codes[500] PASSED [ 90%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_different_status_codes[502] PASSED [ 95%] tests/client/test_card_resolver.py::TestGetAgentCard::test_get_agent_card_returns_agent_card_instance PASSED [100%] ======================== 21 passed, 2 warnings in 0.11s ======================== ``` - [X] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [X] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - [X] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [N/A ] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> 🦕 N/A --------- Co-authored-by: Lukasz Kawka <[email protected]>

…r` (a2aproject#593) # Description Previously, the `JSON-RPC` and `REST` protocols verified agent card signatures after calling `A2ACardResolver.get_agent_card`. This change moves the signature verification logic inside the `A2ACardResolver.get_agent_card` method and adds a unit test to test_card_resolver.py Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly: - [x] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [x] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - Important Prefixes for [release-please](https://github.com/googleapis/release-please): - `fix:` which represents bug fixes, and correlates to a [SemVer](https://semver.org/) patch. - `feat:` represents a new feature, and correlates to a SemVer minor. - `feat!:`, or `fix!:`, `refactor!:`, etc., which represent a breaking change (indicated by the `!`) and will result in a SemVer major. - [x] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [x] Appropriate docs were updated (if necessary)

🤖 I have created a release *beep* *boop* --- ## [0.3.22](a2aproject/a2a-python@v0.3.21...v0.3.22) (2025-12-16) ### Features * Add custom ID generators to `SimpleRequestContextBuilder` ([a2aproject#594](a2aproject#594)) ([04bcafc](a2aproject@04bcafc)) ### Code Refactoring * Move agent card signature verification into `A2ACardResolver` ([6fa6a6c](a2aproject@6fa6a6c)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

# Description Adding 13 tests for server/tasks/id_generator.py They all pass: ``` ============================= test session starts ============================== collecting ... collected 13 items tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_creation_with_all_fields PASSED [ 7%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_creation_with_defaults PASSED [ 15%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_creation_with_partial_fields[kwargs0-task_123-None] PASSED [ 23%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_creation_with_partial_fields[kwargs1-None-context_456] PASSED [ 30%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_mutability PASSED [ 38%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::test_context_validation PASSED [ 46%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::TestIDGenerator::test_cannot_instantiate_abstract_class PASSED [ 53%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::TestIDGenerator::test_subclass_must_implement_generate PASSED [ 61%] tests/server/tasks/test_id_generator.py::TestIDGeneratorContext::TestIDGenerator::test_valid_subclass_implementation PASSED [ 69%] tests/server/tasks/test_id_generator.py::TestUUIDGenerator::test_generate_returns_string PASSED [ 76%] tests/server/tasks/test_id_generator.py::TestUUIDGenerator::test_generate_produces_unique_ids PASSED [ 84%] tests/server/tasks/test_id_generator.py::TestUUIDGenerator::test_generate_works_with_various_contexts[none_context] PASSED [ 92%] tests/server/tasks/test_id_generator.py::TestUUIDGenerator::test_generate_works_with_various_contexts[empty_context] PASSED [100%] ============================== 13 passed in 0.04s ============================== ``` - [X] Follow the [`CONTRIBUTING` Guide](https://github.com/a2aproject/a2a-python/blob/main/CONTRIBUTING.md). - [X] Make your Pull Request title in the <https://www.conventionalcommits.org/> specification. - [X] Ensure the tests and linter pass (Run `bash scripts/format.sh` from the repository root to format) - [N/A] Appropriate docs were updated (if necessary) Fixes #<issue_number_goes_here> 🦕 N/A --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Co-authored-by: Lukasz Kawka <[email protected]>

lkawka

Hi Luca, thanks for the PR; apologies for the delay.

The team agrees with moving to proto-based types. Please address Sam's open comments, but otherwise, it looks good.

Proposed next steps:

Resolve issues and submit to the a2aproject:1.0-a2a_proto_refactor branch.
Merge into 1.0-dev via a separate PR to avoid conflicts with existing implementations.

lkawka · 2026-01-15T11:34:21Z

src/a2a/client/auth/interceptor.py

+    which = scheme.WhichOneof('scheme')
+    if which == 'api_key_security_scheme':
+        return scheme.api_key_security_scheme
+    elif which == 'http_auth_security_scheme':
+        return scheme.http_auth_security_scheme
+    elif which == 'oauth2_security_scheme':
+        return scheme.oauth2_security_scheme
+    elif which == 'open_id_connect_security_scheme':
+        return scheme.open_id_connect_security_scheme
+    elif which == 'mtls_security_scheme':
+        return scheme.mtls_security_scheme
+    return None


lkawka · 2026-01-16T08:38:32Z

src/a2a/utils/task.py

    for part in request.parts:
-        if isinstance(part.root, TextPart) and not part.root.text:
-            raise ValueError('TextPart content cannot be empty')
+        if part.text is not None and not part.text:


lkawka · 2026-01-16T09:22:04Z

src/a2a/utils/constants.py

+TRANSPORT_GRPC = 'GRPC'
+
+
+class TransportProtocol:


This looks like a good candidate for StrEnum

lkawka · 2026-01-16T10:13:22Z

src/a2a/server/request_handlers/default_request_handler.py

-            task_id=params.id,
+            name=params.name,
            push_notification_config=push_notification_config[0],
        )


This commit resolves extensive test failures caused by recent Protocol Buffer changes in a2a_pb2. Key changes: - Removed invalid 'AgentCard.url' references; updated 'RestTransport' and tests to use 'supported_interfaces'. - Updated 'AgentCard' capability assertions to use 'capabilities.extended_agent_card'. - Fixed 'BaseClient.send_message' usage in integration tests to use 'request' kwarg instead of 'message'. - Updated assertion logic in integration tests to check 'received_params.message'. - verified all tests pass including database integration tests. Signed-off-by: Luca Muscariello <[email protected]>

Signed-off-by: Luca Muscariello <[email protected]>

muscariello · 2026-01-16T23:11:43Z

@lkawka I will go through all the remaining comments on Monday. Thanks for the review.

muscariello · 2026-01-17T08:21:16Z

@Tehsmash could you please check again? I am going to recheck all the previous comments on Monday but please recheck as well. I included the latest updates based on the merge queue in the spec based on the latest TSC votes. We'll see on Tue if the spec can be considered complete for v1.
We could also merge this one and propose another PR after to avoid conflicts.

Signed-off-by: Luca Muscariello <[email protected]>

Tehsmash and others added 2 commits November 21, 2025 17:02

[DRAFT] feat: Upgrade A2A to v1.0

b348735

This updates the SDK to be A2A v1.0 compliant, all types are generated from the v1.0 a2a.proto. JSONRPC/HTTP+JSON transports are converted to use the a2a types encoded using ProtoJSON directly from the generated types.

muscariello requested review from a team and a2a-bot as code owners November 28, 2025 21:00

gemini-code-assist bot reviewed Nov 28, 2025

View reviewed changes

muscariello mentioned this pull request Nov 30, 2025

[DRAFT] feat!: Upgrade A2A to v1.0 #556

Closed

4 tasks

muscariello added this to TSC Review Nov 30, 2025

github-project-automation bot moved this to Backlog in TSC Review Nov 30, 2025

Tehsmash suggested changes Dec 1, 2025

View reviewed changes

muscariello marked this pull request as draft December 1, 2025 13:53

Tehsmash suggested changes Dec 1, 2025

View reviewed changes

muscariello changed the title ~~feat!: Migrate from Pydantic types to protobuf-generated types~~ feat!: upgrade SDK to A2A 1.0 specs Dec 1, 2025

Tehsmash reviewed Dec 1, 2025

View reviewed changes

holtskinner changed the title ~~feat!: upgrade SDK to A2A 1.0 specs~~ refactor!: upgrade SDK to A2A 1.0 specs Dec 1, 2025

muscariello added 3 commits December 1, 2025 20:04

chore: remove AIP-discussion-response.md from tracking

ac1050d

Tehsmash reviewed Dec 2, 2025

View reviewed changes

holtskinner changed the base branch from main to 1.0-a2a_proto_refactor December 2, 2025 16:49

holtskinner requested a review from lkawka December 2, 2025 16:50

lkawka and others added 2 commits December 3, 2025 15:52

didier-durand and others added 3 commits December 12, 2025 14:12

chore: Fixing typos (final round) (a2aproject#588)

090ca9c

# Description Read further and discovered this additional (and final) set of typos

a2a-bot and others added 8 commits December 12, 2025 18:04

kabir mentioned this pull request Jan 6, 2026

feat: Fix capabilities tests for 1.0 a2aproject/a2a-tck#108

Merged

This was referenced Jan 9, 2026

fix: historyLength=0 returns full history (#573) #612

Open

fix: task state from/to proto missing reject #622

Open

lkawka reviewed Jan 16, 2026

View reviewed changes

muscariello added 2 commits January 16, 2026 23:14

chore: merge main and fix tests for proto refactor

a680c18

Signed-off-by: Luca Muscariello <[email protected]>

muscariello force-pushed the a2a_proto_refactor branch from 601ef0b to a680c18 Compare January 16, 2026 23:10

muscariello marked this pull request as ready for review January 16, 2026 23:10

style: fix spelling errors and add words to allow list

0ae8548

muscariello requested a review from Tehsmash January 17, 2026 08:18

muscariello added 4 commits January 17, 2026 09:26

style: add more tech terms to spelling allow list

bd552f6

style: fix spelling of interruptible

d8df048

build(spelling): exclude generated protobuf types from spell check

7433e04

fix: replace non-inclusive language in optionals.py

8e5ab33

Signed-off-by: Luca Muscariello <[email protected]>

muscariello force-pushed the a2a_proto_refactor branch from 67f27a8 to 8e5ab33 Compare January 17, 2026 15:59

	if part.text is not None and not part.text:
	if part.HasField('text') and not part.text:

		del task_copy.history[:]
		task_copy.history.extend(limited_history)

	del task_copy.history[:]
	task_copy.history.extend(limited_history)
	task_copy.history[:] = limited_history



		def _get_security_scheme_value(scheme: SecurityScheme):
		def _get_security_scheme_value(scheme: SecurityScheme) -> _SecuritySchemeValue:

		TRANSPORT_GRPC = 'GRPC'


		class TransportProtocol:

refactor!: upgrade SDK to A2A 1.0 specs #572

Are you sure you want to change the base?

refactor!: upgrade SDK to A2A 1.0 specs #572

Uh oh!

Conversation

muscariello commented Nov 28, 2025 • edited by holtskinner Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Breaking Changes

Changes

Testing

Related

Uh oh!

gemini-code-assist bot commented Nov 28, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

muscariello commented Nov 28, 2025 •

edited by holtskinner

Loading